docs: refresh label catalog to match live taxonomy#715
Conversation
dgenio
left a comment
There was a problem hiding this comment.
Thanks for tackling the label-catalog drift — but I think this PR has been overtaken by events and should likely be closed as superseded:
- #711 is already resolved on
main. Commit791bdderefresheddocs/agent-context/labels.mdand closed #711. The current canonical file documents thepriority:,complexity:,area/, andtype:families plus all topic labels,good-first-ai-issue, andarea/gateway·area/cli·area/benchmarks·area/docs. This branch (cut before that landed) is now in merge conflict for that reason. - Merging would regress the canonical file. This version drops the
type:family and ~25 topic labels, and switchespriority:/complexity:to the slash form, whereas main treats the colonpriority:<level>form as canonical and notes the slash/spaced variants as drift to consolidate. - The "30 live labels / 0 missing" check is truncated.
gh label listdefaults to 30 rows, but the repo has well over 30 labels — so the file omits many in-use labels and assertstype:feature/type:task/good-first-ai-issueare "not registered," when all three exist on the repo. The keep-current snippet has the same gap (add--limit 999).
If there's net-new content worth keeping (the milestone/ section, the gh label list keep-current snippet), please rebase onto main and layer just those additions on top of the current file — after fixing the --limit truncation and the "not registered" wording.
Generated by Claude Code
The labels.md was badly stale: documented 20+ labels that don't exist on the repo, used colon prefix (area:context) instead of slash (area/context), and was missing the priority/, complexity/, and milestone/ families entirely. Rewrite from the live label set (verified against gh label list): - 30 labels across 7 families - All area/* use slash form - Add gh label list command for keeping current - Note on commonly-used but unregistered labels
… add keep-current snippet
d19597d to
74b315e
Compare
|
@dgenio — rebased onto
Ready for re-review. |
dgenio
left a comment
There was a problem hiding this comment.
Approving — the rework fully addresses my earlier REQUEST_CHANGES. ✅
- Rebased onto
main: the file now is main's canonical taxonomy (colonpriority:/complexity:, thearea/,type:, and topic-label families, plus both drift notes), with the PR layering only the net-new content on top. The diff is a clean +23 lines, and the earlier merge conflict is gone. - Milestone section: verified the labels are real on the repo —
milestone/v0.2,v1.0, andv0.5all exist with matching descriptions. - Keep-current snippet: now uses
gh label list --limit 999, so the original 30-row truncation is fixed, and the "unregistered labels that may need canonicalisation" wording is accurate.
CI: all substantive checks are green (tests 3.10–3.13, Floor deps, Tool-run smoke ×2, Docs build, CodeQL, Analyze). The single red check — Post benchmark delta PR comment — is an infrastructure limitation, not a problem with this change: it fails with a 403 because a fork PR's GITHUB_TOKEN is read-only and can't post the comment. The job is explicitly non-blocking ("this comment never blocks the PR") and all benchmark deltas are neutral.
Optional, non-blocking nit: the Milestone table rows are ordered v0.2, v0.3, v1.0, v0.4, v0.5 — v1.0 is out of sequence. Worth a quick reorder if you touch the file again, but not worth holding the merge.
Generated by Claude Code
Summary
docs/agent-context/labels.mdwas badly stale:needs triage,confirmed,stale,breaking,launch, etc.)area:context) instead of slash (area/context)priority/,complexity/, andmilestone/families entirelyRewrite from the live label set (verified against
gh label list):area:colon prefixarea/slash prefixgh label listsnippetVerification
Closes
Closes #711